Skip to content

Comments

Bumped dependancy versions in the lock file#303

Merged
feldlime merged 8 commits intomainfrom
bump_lock_dependancy_versions
Feb 20, 2026
Merged

Bumped dependancy versions in the lock file#303
feldlime merged 8 commits intomainfrom
bump_lock_dependancy_versions

Conversation

@feldlime
Copy link
Collaborator

@feldlime feldlime commented Feb 10, 2026 •
edited
Loading

Description

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Optimization

How Has This Been Tested?

Before submitting a PR, please check yourself against the following list. It would save us quite a lot of time.

  • Have you read the contribution guide?
  • Have you updated the relevant docstrings? We're using Numpy format, please double-check yourself
  • Does your change require any new tests?
  • Have you updated the changelog file?

@feldlime feldlime requested a review from Copilot February 10, 2026 11:49
Copilot AI reviewed Feb 10, 2026
View reviewed changes
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates SASRec transformer unit tests to reflect changed recommendation ordering/output, likely as a consequence of dependency version changes.

Changes:

  • Adjust expected Columns.Item sequences in SASRec recommendation tests (test_u2i_with_key_and_attn_masks, test_u2i_with_item_features).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov
Copy link

codecov bot commented Feb 15, 2026 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (9b3992e) to head (7a47f23).
⚠️ Report is 125 commits behind head on main.

Additional details and impacted files 
@@             Coverage Diff             @@
##              main      #303     +/-   ##
===========================================
  Coverage   100.00%   100.00%             
===========================================
  Files           45        85     +40     
  Lines         2242      5874   +3632     
===========================================
+ Hits          2242      5874   +3632

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@feldlime feldlime marked this pull request as ready for review February 20, 2026 08:22
Copilot AI reviewed Feb 20, 2026
View reviewed changes
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 5 changed files in this pull request and generated 4 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@feldlime feldlime merged commit f2aa6ef into main Feb 20, 2026
16 checks passed
@feldlime feldlime deleted the bump_lock_dependancy_versions branch February 20, 2026 13:58
This was referenced Feb 20, 2026
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@blondered blondered blondered approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@feldlime @blondered